Codecov Report

Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Project coverage is 60.09%. Comparing base (bbcb2f6) to head (6e9287d).
Report is 172 commits behind head on master.

@@            Coverage Diff             @@
##           master    #2670      +/-   ##
==========================================
- Coverage   60.11%   60.09%   -0.03%     
==========================================
  Files         560      560              
  Lines       74918    74925       +7     
==========================================
- Hits        45039    45027      -12     
- Misses      26504    26519      +15     
- Partials     3375     3379       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Hi @leohhhn , when you have a bit of time, could you check my code please

For the moment, I've put my PR in draft form because I don't think my approach is the right one for these reasons:

• Iteration on all accounts
• No persistence for the total result of the part. So every time the function is called, everything is recalculated.

I realised this when I saw the PR for @thinhnx-var

I did not see this PR after I looked at the original issue again. Thats why I pushed my PR @DIGIX666. 🙏
Btw, I see that the things you commented above is true. Because we should consider about the cost of calculating total coins.

I did not see this PR after I looked at the original issue again. Thats why I pushed my PR @DIGIX666. 🙏
Btw, I see that the things you commented above is true. Because we should consider about the cost of calculating total coins.

No problem, I should have asked if someone was working on it first ;)

This PR is stale because it has been open 3 months with no activity. Remove stale label or comment or this will be closed in 3 months.